Skip to content

In-game keybind editor - #8153

Open
burnhamrobertp wants to merge 74 commits into
beyond-all-reason:masterfrom
burnhamrobertp:feature/396-keybind-editor
Open

In-game keybind editor#8153
burnhamrobertp wants to merge 74 commits into
beyond-all-reason:masterfrom
burnhamrobertp:feature/396-keybind-editor

Conversation

@burnhamrobertp

@burnhamrobertp burnhamrobertp commented Jun 30, 2026

Copy link
Copy Markdown
Contributor

Replaces the "info" tab with a keybind editor as the first tab of the Keys panel. Closes #396.

The shipped presets were uikeys files that included one another, so nothing outside the game could read them. They're now data, alongside a catalog of what's editable, so Chobby and the new lobby can build against the same contract. Catalog and profiles stay separate because a catalog entry is per action while a binding is per action and profile - the four shipped profiles share only 207 of the 381 actions they bind between them.

Two engine details explain most of the shape. keyreload clears the keymap before it loads, so a profile defines every binding rather than a delta. And two actions on one keyset are tried in the order they were bound, so an edit rewrites a binding where it sits instead of re-adding it at the end. Separately, the panel takes text ownership while it's open, or actionHandler fires the keybind you're editing as you press it.

Legacy still ships, so which build menu a profile implies is inferred from its bindings; that goes when Legacy does.

Updated video of the keybind editor UI/UX can be found in discord here.

8mb.video-71i-P3vAN9PQ.mp4

AI disclosure: written with assistance from Claude Code.

@github-actions

github-actions Bot commented Jun 30, 2026

Copy link
Copy Markdown
Contributor

Integration Test Results

15 tests  ±0   7 ✅ ±0   4s ⏱️ ±0s
 1 suites ±0   8 💤 ±0 
 1 files   ±0   0 ❌ ±0 

Results for commit 298e58f. ± Comparison against base commit 95ba962.

♻️ This comment has been updated with latest results.

@burnhamrobertp

Copy link
Copy Markdown
Contributor Author

(was built on top of #8143 and so is left in a draft state until that merges)

@burnhamrobertp
burnhamrobertp force-pushed the feature/396-keybind-editor branch from e0a5c38 to c85bfa9 Compare July 1, 2026 19:35
@burnhamrobertp
burnhamrobertp marked this pull request as ready for review July 9, 2026 06:24
@WatchTheFort

Copy link
Copy Markdown
Member

The removal of the info panel has orphaned some localization strings; I'm not sure the process on cleaning those out end-to-end and whether or not that's as simple as removing them from the localizations file?

Delete them from the English source files, Transifex will take care of deleting them in other languages.

@burnhamrobertp

burnhamrobertp commented Jul 15, 2026

Copy link
Copy Markdown
Contributor Author

Per review feedback, with the removal of the other tabs:

Image removed as no longer really relevant

Comment thread luaui/Include/keybind_editor_view.lua
@burnhamrobertp

Copy link
Copy Markdown
Contributor Author

Here marks the beginning of work refactoring the UX/implementation to meet WTF's stated desires during today's review session

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

For the absolutely enormous number of orders we have, they should be split out into a dedicated translation file.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Enhancement New feature or request UI Everything related UI, GUI, etc.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Key Binding Menu

3 participants